test-concurrency: Don't lower timeout
authorColin Walters <walters@verbum.org>
Tue, 13 Jun 2023 20:17:16 +0000 (16:17 -0400)
committerColin Walters <walters@verbum.org>
Tue, 13 Jun 2023 20:17:16 +0000 (16:17 -0400)
commit7892818fb183e9b510e920a6f0658a214e4aa173
tree802492a7b177df3f27e4eda04fe72cdc16935efd
parent92445188960d03d5b8dd1ad681beb776611984b2
test-concurrency: Don't lower timeout

I think there's never been a real race condition here.  Instead
the problem is:

- We have a timeout on the lock, after which we error out (30s)
- This test actually *lowers* the timeout

Looking through the failures for test-concurrency what I see
is
`error: Locking repo exclusive failed: Resource temporarily unavailable`
which is us hitting the timeout.

Hardcoded timeouts are just going to be subject to race conditions.
I understand not wanting to block forever in some cases, but any
arbitrary timeout is just going to get hit in real world conditions
too.

Anyways for now, stop shooting ourselves in the foot and at least
keep the timeout at the default.

Closes: https://github.com/ostreedev/ostree/issues/2038
tests/test-concurrency.py